Add conditional stack traces for local and development debugging and keeps the existing incidentIdentifier behavior used for log discovery.

Stack traces are now included in CdaError.details.stackTrace when the request is
explicitly configured for trace output:

• request is on localhost (docker compose)
  OR
• request is authenticated as a CWMS User Admins AND in dev environment (env.lower().includes('dev'))

The implementation keeps CdaError as a response DTO and places the request-aware policy
in ErrorTraceSupport.

To make that behavior apply consistently, this PR also removes several redundant
controller-level catch (Exception) wrappers that were bypassing ApiServlet, and
updates remaining endpoint 500 handlers to make their responses through
ErrorTraceSupport.

Partly fixes some of

• Improve exception handling #1625
  but not the ConnectionPreparingDataSource

Other potentially related issues include:

• Improve Exception/Error handling logic, heirarchy, etc #1106
• Be more principled about Exception handling #299
• JooqDao changes removed exception wrapping #350
• Cannot retrieve Elev Timeseries #1342 (helps with diagnosing/better traceback visibility)

Trying to centralize general errors where possible and reduce fragmentation

Not trying to solve a new base exception direction, or a better inheritance model mainly because @MikeNeilson said to discuss more before doing that in another issue.